Skip to content

feat: distribute ozctl CLI binary for Linux#438

Merged
diranged merged 3 commits intomainfrom
feat/ozctl-linux-build
Mar 12, 2026
Merged

feat: distribute ozctl CLI binary for Linux#438
diranged merged 3 commits intomainfrom
feat/ozctl-linux-build

Conversation

@dnguy078
Copy link
Copy Markdown
Collaborator

@dnguy078 dnguy078 commented Mar 3, 2026

Summary

  • Add linux to the CLI build's goos list in .goreleaser.yml so ozctl is distributed for Linux (amd64 + arm64)
  • Add explicit archives section scoped to the CLI build only, so the manager binary (which is only used in the Docker image) is not included in release archives
  • Enables automated installation of ozctl on Linux environments (e.g. CI, cloud dev environments)

Test plan

  • Verify goreleaser config is valid (goreleaser check)
  • Confirm CI build passes with separate archive config
  • E2e tests pass across all supported Kubernetes versions

🤖 Generated with Claude Code

Add linux to the cli build's goos list so ozctl is included in the
linux_amd64 and linux_arm64 release tarballs alongside manager.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the repo label Mar 3, 2026
@dnguy078 dnguy078 marked this pull request as ready for review March 3, 2026 23:52
@dnguy078 dnguy078 requested a review from diranged as a code owner March 3, 2026 23:52
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 3, 2026

Pull Request Test Coverage Report for Build 22974718385

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 32 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.1%) to 35.907%

Files with Coverage Reduction New Missed Lines %
internal/cmd/manager/main.go 32 0.0%
Totals Coverage Status
Change from base Build 21759598336: -0.1%
Covered Lines: 1037
Relevant Lines: 2888

💛 - Coveralls

@diranged
Copy link
Copy Markdown
Owner

diranged commented Mar 6, 2026

@dnguy078 its failing to pass the build..

@diranged
Copy link
Copy Markdown
Owner

@dnguy078 going to tackle the build failure?

dnguy078 and others added 2 commits March 11, 2026 14:11
…ary count

The addition of Linux to the CLI build caused goreleaser to fail because
the default archive had different binary counts per platform (Linux has
both manager + ozctl, macOS only has ozctl). Splitting into separate
archives resolves this.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The manager binary is only used in the Docker image and doesn't need
its own release archive.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dnguy078
Copy link
Copy Markdown
Collaborator Author

ah yea sorry forgot about this. Fixed

@diranged diranged merged commit fe5faeb into main Mar 12, 2026
19 checks passed
@diranged diranged deleted the feat/ozctl-linux-build branch March 12, 2026 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants